-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Apply new ruff rules #139
Apply new ruff rules #139
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #139 +/- ##
==========================================
- Coverage 65.12% 65.10% -0.03%
==========================================
Files 25 25
Lines 2644 2642 -2
Branches 417 415 -2
==========================================
- Hits 1722 1720 -2
Misses 811 811
Partials 111 111 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
ac4ff83
to
c9a6902
Compare
C409 Unnecessary list comprehension passed to `tuple()` (rewrite as a generator)
C419 Unnecessary list comprehension
UP008 Use `super()` instead of `super(__class__, self)`
RUF022 `__all__` is not sorted
RUF023 `__slots__` is not sorted
PERF401 Use a list comprehension to create a transformed list
FURB188 Prefer `removeprefix` over conditionally replacing with slice.
SIM401 Use `.get()` instead of an `if` block
ISC001 Implicitly concatenated string literals on one line
c9a6902
to
58fbf88
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm. Small suggestion while we're looking at it.
Co-authored-by: Chris Markiewicz <[email protected]>
Co-authored-by: Chris Markiewicz <[email protected]>
24.1.0 (December 18, 2024) New feature release in the 24.1.x series. This release includes a migration of most if not all reporting interfaces from NiWorkflows. This release also supports Python 3.13 and Numpy 2. * ENH: Finalize migration of reporting interfaces (#71) * ENH: Allow figures in session folder (#138) * RF: Replace nireports.data.Loader with acres.Loader (#142) * STY: Apply new ruff rules (#139) * MAINT: Add tox.ini, test minimum dependencies (#141)
No description provided.